Skip to content

Added cooldown to sign in button#120

Merged
abrar-amin merged 2 commits intomainfrom
Abrar-Add-Cooldown
Apr 8, 2026
Merged

Added cooldown to sign in button#120
abrar-amin merged 2 commits intomainfrom
Abrar-Add-Cooldown

Conversation

@abrar-amin
Copy link
Copy Markdown
Collaborator

Summary

Added a 15 second cooldown to the send sign-in email link button.

Test Plan

Tested in developer sandbox

Notes

This is purely a frontend change.

Simulator.Screen.Recording.-.iPhone.17.-.2026-04-06.at.14.55.05.mov

@abrar-amin abrar-amin requested review from clementroze and laylaliuuu and removed request for laylaliuuu April 6, 2026 18:58
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 6, 2026

Deploy Preview for redi-love ready!

Name Link
🔨 Latest commit 88c1d51
🔍 Latest deploy log https://app.netlify.com/projects/redi-love/deploys/69d5ae02dcc1a2000898fb3b
😎 Deploy Preview https://deploy-preview-120--redi-love.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Collaborator

@clementroze clementroze left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Abrar! ⏲️

<View style={styles.buttonContainer}>
<Button
title="Send Sign-In Link"
title={cooldownSeconds > 0 ? `Resend in ${cooldownSeconds}s` : 'Send Sign-In Link'}
Copy link
Copy Markdown
Collaborator

@clementroze clementroze Apr 6, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is really minor, but to prevent flickering of the numbers + the label changing width, do u wanna try adding the font-variant-numeric: tabular-nums; CSS property?

See this SO post for more details

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 8, 2026

Copy link
Copy Markdown
Collaborator

@jacquelinecai jacquelinecai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yessir

@abrar-amin abrar-amin merged commit 57026ac into main Apr 8, 2026
11 checks passed
@abrar-amin abrar-amin deleted the Abrar-Add-Cooldown branch April 8, 2026 01:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants